-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make XOccupiedToUnoccupiedDelay attributes in OccupancySensing cluster managed by AAI #36777
base: master
Are you sure you want to change the base?
Conversation
Changed Files
|
PR #36777: Size comparison from 5d42d92 to 35d065b Full report (69 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #36777: Size comparison from 5d42d92 to 736a7e8 Full report (69 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #36777: Size comparison from 9e203e2 to 62ef0b6 Full report (69 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wqx6 please add a Testing
section to describe how this change was tested.
Since I see no unit test changes, this may be "manual" in which case please describe testing in more detail (what commands were run, what output existed before and what is the new behavior) as well as a justification why this change is impossible to test automatically.
We are managing the
*OccupiedToUnoccupiedDelay
attributes in the Write function of AAI for OccupancySensing cluster but these attributes are not added in theattributeAccessInterfaceAttributes
in thezcl.json
. So make all these attributes managed by AAI.